Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Urban opt50 #615

Merged
merged 88 commits into from
Feb 1, 2021
Merged

Urban opt50 #615

merged 88 commits into from
Feb 1, 2021

Conversation

brianlball
Copy link
Contributor

No description provided.

…oint

convert create_data_point_filename: 'create_data_point.rb' to create_and_run_datapoint_filename: 'create_and_run_datapoint_uniquegroups.R'
add check for it in nsga_nrel.rb
add boundary check to morris.R
add create_and_run_datapoint_urbanopt.R
add urbanopt flag to DataPoint and Analysis
add --UO run option to api_create_datapoint.rb
add uo_debug branch for UO gems
add SEB_LHS test
options for variable
@nllong
Copy link
Member

nllong commented Jan 15, 2021

Nice! I'll take a look at this early next week!

@tijcolem
Copy link
Contributor

@brianlball Can you merge in the latest develop branch? We enabled Github actions and disabled Travis. You can add all the algo tests in now as we have more run time allotted.

@brianlball
Copy link
Contributor Author

@tijcolem merged in develop, green. this still ran the tests in this dir, correct? https://github.com/NREL/OpenStudio-server/tree/develop/server/spec/features

Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I'll merge as soon as the tests pass.

@nllong
Copy link
Member

nllong commented Feb 1, 2021

@tijcolem merged in develop, green. this still ran the tests in this dir, correct? https://github.com/NREL/OpenStudio-server/tree/develop/server/spec/features

Looks like the urbanopt tests run within docker! Should be good to go. I have one more tweak that I'm going to commit.

image

@brianlball
Copy link
Contributor Author

Damn thats a beautiful sight....

@nllong
Copy link
Member

nllong commented Feb 1, 2021

I have one last commit that is nothing more than a couple typos... then I'll merge down.

@nllong nllong merged commit 7aa10d3 into develop Feb 1, 2021
@nllong nllong deleted the UrbanOpt50 branch February 1, 2021 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants